Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds --plays option to add play count #183

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

alexgorbatchev
Copy link

Why? Because sometimes when you want to archive a very long play list (eg https://soundcloud.com/muloka/sets/burning-man-2016) it could be handy to be able to sort files by "popularity". To achieve that, --plays option adds play count at the moment of the download as the first thing in the file name.

If you want to resync the playlist, files that were downloaded already won't be redownloaded with new play count. At the same time, existing file's play count won't be updated either.

I don't know if the feature would be interesting to merge, but I thought I would share it anyways.

@Miserlou
Copy link
Owner

Good idea but Travis isn't happy..

@alexgorbatchev
Copy link
Author

alexgorbatchev commented Jul 10, 2017

I fixed the tests that my PR broke. Other failures currently happen on master now :(

@muloka
Copy link

muloka commented Sep 19, 2017

…ownloading from SoundCloud playlists.

Why? Because sometimes when you want to archive a very long play list (eg https://soundcloud.com/muloka/sets/burning-man-2016) it could be handy to be able to sort files by "popularity". To achieve that, `--plays` option adds play count at the moment of the download as the first thing in the file name.

If you want to resync the playlist, files that were downloaded already won't be redownloaded with new play count. At the same time, existing file's play count won't be updated either.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants